Skip to content

Add lint:fix Commands and Update Agent Guidance#1329

Open
jamesricky wants to merge 3 commits intomainfrom
agents-fix-codestyle
Open

Add lint:fix Commands and Update Agent Guidance#1329
jamesricky wants to merge 3 commits intomainfrom
agents-fix-codestyle

Conversation

@jamesricky
Copy link
Contributor

Description

Previously there was no consistent way to auto-fix linting and formatting issues — only lint check commands existed.

This adds lint:fix scripts to api, admin, and site, as well as a root-level script that runs all three in parallel.
Each script wires together ESLint --fix, Prettier --write, and (for site) stylelint --fix.
AGENTS.md is updated to document these commands and to instruct agents to run lint:fix after making code changes, keeping the codebase consistently formatted without manual intervention.

@jamesricky jamesricky self-assigned this Mar 6, 2026
@jamesricky jamesricky requested a review from thomasdax98 March 6, 2026 14:22
@auto-assign auto-assign bot requested a review from johnnyomair March 6, 2026 14:22
@jamesricky jamesricky changed the title Add lint:fix Commands and Update Agent Guidance Add lint:fix Commands and Update Agent Guidance Mar 9, 2026
johnnyomair
johnnyomair previously approved these changes Mar 10, 2026
Copy link
Collaborator

@johnnyomair johnnyomair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should add this in core as well.

@jamesricky
Copy link
Contributor Author

We should add this in core as well.

👍🏼 I'll create a PR

@jamesricky
Copy link
Contributor Author

We should add this in core as well.

vivid-planet/comet#5303

thomasdax98
thomasdax98 previously approved these changes Mar 13, 2026
@thomasdax98 thomasdax98 dismissed their stale review March 13, 2026 08:36

we should also recycle the eslint script here (see vivid-planet/comet#5303 (comment))

Copy link
Collaborator

@johnnyomair johnnyomair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we add a lint:fix script for create-app as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants